Skip to content

Fix Claude onboarding terminal popup#2078

Merged
wesbillman merged 1 commit into
mainfrom
pinky/claude-headless-login
Jul 18, 2026
Merged

Fix Claude onboarding terminal popup#2078
wesbillman merged 1 commit into
mainfrom
pinky/claude-headless-login

Conversation

@wesbillman

Copy link
Copy Markdown
Collaborator

Summary

  • route both current and legacy Claude subscription auth method IDs through the existing headless login process
  • prevent first-launch Claude setup from opening a visible terminal
  • keep Codex and non-subscription terminal auth behavior unchanged

Root cause

The current @agentclientprotocol/claude-agent-acp advertises claude-ai-login, while Buzz only special-cased the legacy claude-login ID. The current payload therefore fell through to visible terminal launch even though its OAuth flow opened the browser successfully.

Verification

  • focused native auth tests: 15 passed
  • full desktop Tauri suite: 1,469 passed, 13 ignored; 3 diagnostics passed
  • pre-push: branch skew, desktop checks/tests, desktop Tauri tests, mobile tests, and Rust tests all passed
  • formatting and diff checks passed

Route both supported Claude subscription auth method IDs through the existing
headless login process so first launch opens the browser without exposing a
terminal. Keep non-subscription terminal methods unchanged and cover both
adapter payloads.

Co-authored-by: Pinky <44b8e82baa6e0e254e0208d68f335c283c94e7b78dd1fa10d5a49d3f13dd0435@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: Wes <wesbillman@users.noreply.github.com>
@wesbillman
wesbillman requested a review from a team as a code owner July 18, 2026 17:49
@wesbillman
wesbillman enabled auto-merge (squash) July 18, 2026 17:50
@wesbillman
wesbillman merged commit f125f08 into main Jul 18, 2026
25 checks passed
@wesbillman
wesbillman deleted the pinky/claude-headless-login branch July 18, 2026 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants